Extend APS orchestration CLI#40
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds an orchestration layer to the APS bash CLI to help users progress work items (next, start, complete) and inspect dependency structure (graph), including generation of ephemeral context packages for in-progress work.
Changes:
- Add
aps start,aps complete, andaps graphcommands plus context package generation (.aps/context/<ID>.md). - Update scaffolding to install orchestration support files and ignore generated context output.
- Expand test suite/fixtures to cover orchestration flows and init-time installation behavior.
Reviewed changes
Copilot reviewed 7 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
bin/aps |
Wires new orchestration subcommands into the CLI and updates help text/examples. |
lib/orchestrate.sh |
Implements start/complete/graph and context package generation + markdown mutation helpers. |
lib/scaffold.sh |
Ensures orchestration library is installed and .aps/context is ignored on init; updates migrate cleanup list. |
test/run.sh |
Adds orchestration tests and verifies init installs orchestration support + ignore rules. |
test/orchestrate.sh |
Adds end-to-end tests for next/start/complete/graph including context regeneration behavior. |
test/fixtures/orchestrate/plans/modules/core.aps.md |
Adjusts fixture module/item status + adds a Learning used by dependency-learning extraction tests. |
test/fixtures/orchestrate/plans/modules/auth.aps.md |
Adds Files field for context packaging and adds a “final item before decisions” edge-case fixture. |
plans/modules/orchestrate.aps.md |
Updates orchestration work item statuses to Complete. |
.gitignore |
Ignores generated .aps/context/ packages in the repo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| orch_emit_section "$file" "Interfaces" | ||
| echo | ||
| echo "## Decisions" | ||
| orch_emit_section "$file" "Decisions" || true |
Comment on lines
+435
to
+440
| if (mode == "learning") { | ||
| learning_line = "- **Learning:** \"" value "\"" | ||
| if (validation_idx >= 0) { | ||
| insert_idx = validation_idx | ||
| # advance past any multi-line continuation under Validation | ||
| while (insert_idx + 1 < bcount && continuation_line(insert_idx + 1)) insert_idx++ |
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
aps start,aps complete, andaps graphto the bash CLI orchestration layer..aps/context/<ID>.mdpackages on start, including module scope, dependency learnings, and related files.APS Work Items
aps nextcommand (Complete)aps startandaps complete(Complete)aps graph(Complete)Review
Test Plan
bash test/run.shnpx markdownlint-cli "**/*.md"